Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Building Completion #52

Merged
merged 9 commits into from
Mar 22, 2022
Merged

Add Building Completion #52

merged 9 commits into from
Mar 22, 2022

Conversation

CharlesGaydon
Copy link
Collaborator

...for low-density areas close to confirmed buildings

…counted for

This happens in open pull requests when new commits are pushed to the merged branch.
See actions/checkout#299
@CharlesGaydon
Copy link
Collaborator Author

@MichelDaab FYI, il y a un soucis dans le workflow git ; "Full module run on LAS subset" lance la commande docker par defaut tel que définie dans le Dockerfile d'une précédente version du containeur :
https://github.com/IGNF/lidar-prod-quality-control/runs/5643422789?check_suite_focus=true
On lit dans la configuration printée :

│       codes:                                                                  
│         candidates:                                                           
│           building:                                                           
│           - 19                                                                
│           - 20                                                                
│           - 110                                                               
│           - 112                                                               
│           - 114                                                               
│           - 115   

Alors que dans le dockerfile, je corrige ces codes par le commit suivant (qui est inclu dans cette pull request) :
b01fd07

Une preuve encore plus nette c'est que dans les logs de cette commande, il n'y a pas le message lié à la "Building Completion" supposé être rajouté

[2022-03-22 13:07:32,852][lidar_prod.tasks.building_validation][INFO] - Using best trial from: /CICD_github_assets/M8.0/20220204_building_val_V0.0_model/M8.0B2V0.0_buildingvalidation_thresholds.pickle
[2022-03-22 13:07:32,853][lidar_prod.tasks.building_validation][INFO] - Applying Building Validation to file 
/CICD_github_assets/M8.0/20220204_building_val_V0.0_model/subsets/871000_6617000_subset_with_probas.las
[2022-03-22 13:07:32,854][lidar_prod.tasks.building_validation][INFO] - Preparation - Clustering + Requesting Building database
[2022-03-22 13:07:47,868][lidar_prod.tasks.building_validation][INFO] - Using AI and Databases to update cloud Classification
[2022-03-22 13:07:48,040][lidar_prod.tasks.building_validation][INFO] - Saved to
/CICD_github_assets/app/871000_6617000_subset_with_probas.las
[2022-03-22 13:07:48,064][lidar_prod.tasks.building_identification][INFO] - Applying Building Identification to file 
/CICD_github_assets/app/871000_6617000_subset_with_probas.las
[2022-03-22 13:07:48,064][lidar_prod.tasks.building_identification][INFO] - Clustering of points with high building proba
[2022-03-22 13:07:49,664][lidar_prod.utils.utils][INFO] - Processing time of apply: 16.84s

Alors qu'on devrait voir afficher les msg de log définis dans la méthode run dans b816349#diff-4a69216b6e20c91c38e709374b085bd754aea04f8197377cc27c446754559e10.

J'ai tenté de passer le checkout de V2 à V1 (cf. dernier commit). Car ca a l'air d'être lié aux actions git, il y a des issues dans lesquelles je ne me suis pas plongé... (p.ex. actions/checkout#299)

Je ne sais pas si c'est ici un problème de checkout, ou bien un soucis avec la version du containeur créée.

@CharlesGaydon
Copy link
Collaborator Author

Ok, je viens de trouver une cause potentielle : il y a deux noms d'image docker qui coexistent dans le workflow : tu crée une image lidar_prod_im, mais tu utilise predict_im, qui n'avait donc pas été renommée, et ça doit donc taper dans une vieille image.

Je corrige et je vois ce que ça donne.

@CharlesGaydon
Copy link
Collaborator Author

Corrigé !

@CharlesGaydon CharlesGaydon merged commit 272d221 into main Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant